Add onEndTimeChange event (#838)#845
Conversation
|
@developer-ocansey — Flagging this for your review. Recommendation: Worth porting to the current TypeScript codebase What it doesAdds an Why it's valuableThis is a commonly requested feature (linked to issue #838). The current codebase has no equivalent. The implementation is clean with proper separation of concerns:
What needs adaptationThe PR targets the old JS files and has merge conflicts. The current
Effort estimateMedium — the concept maps directly to the current architecture, just needs TS conversion and PointerEvents adaptation. |
|
Thank you for this contribution! Unfortunately the codebase has been substantially rewritten since this PR was opened (full TypeScript migration, new test suite, and various architectural changes). This PR is now too outdated to merge cleanly. Please feel free to reimplement this feature against the latest |
Issue Number
#838
Overview of PR
I indeed didn't found a good solution to this. So I decided to add an event which is called when scrolling ends. This way I can reload data when this new event gets fired. I think this resolves the issue for me.